Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add logging #28

Merged
merged 1 commit into from
Feb 14, 2024
Merged

Add logging #28

merged 1 commit into from
Feb 14, 2024

Conversation

tillywoodfield
Copy link
Collaborator

Use logging instead of print statements to improve the usability of logs:

  • Add timestamp to all logs
  • Include iatikit logs

Sample log output:

2024-02-14T09:27:42:INFO:iatidata:Creating activities table
2024-02-14T09:27:42:INFO:iatidata:Getting standard
2024-02-14T09:27:43:INFO:iatikit.utils.download:Downloading IATI Standard schemas...
2024-02-14T09:27:59:INFO:iatikit.utils.download:Downloading IATI Standard codelists...
2024-02-14T09:29:20:INFO:iatikit.utils.download:Downloading IATI Standard codelist mappings...
2024-02-14T09:29:23:INFO:iatidata:Getting regisitry data
2024-02-14T09:29:25:INFO:iatikit.utils.download:Downloading all IATI registry data...
2024-02-14T09:33:18:INFO:iatikit.utils.download:Unzipping data...
2024-02-14T09:33:30:INFO:iatikit.utils.download:Cleaning up...
2024-02-14T09:33:30:INFO:iatidata:Loading registry data into database...
...

@tillywoodfield tillywoodfield self-assigned this Feb 14, 2024
@odscjames odscjames merged commit b3ee8a6 into main Feb 14, 2024
4 checks passed
@odscjames odscjames deleted the logging branch February 14, 2024 14:13
Copy link
Collaborator

@odscjames odscjames left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approve

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants